-
Notifications
You must be signed in to change notification settings - Fork 209
Conversation
Is it as it looks in the screenshot? Because that looks really stretched @gaurivn |
@gaurivn can you squash the 2 commits? |
Yes, I took the screenshot of running website after I cropped out Systers' logo. |
Sure |
Some changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gaurivn the logo looks a bit stretched, can you take a higher resolution image :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should use a better resolution image or use something like .webp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll see if I can look for a better resolution image. I'll link the file in the comments. If it isn't available we'll see for some other alternative, it currently looks really stretched.
@gaurivn Were you able to find better resolution logo? |
Sorry for the delay, I got a few AnitaB.org logos - better resolution ones, I'll try with them. |
@gaurivn awesome. Sure |
@gaurivn Any updates? |
Can you add a photo of how it looks now? |
I think I don't have the bandwidth for this atm..faced some issues while trying to run again after changing logos.. closing this PR and making it available. |
Description
Removed Systers' Logo
Fixes #546
Type of Change:
Delete irrelevant options.
How Has This Been Tested?
Describe the tests you ran to verify your changes. Provide instructions or GIFs so we can reproduce. List any relevant details for your test.
Checklist:
Delete irrelevant options.